Skip to content

Conversation

@janvogt
Copy link

@janvogt janvogt commented Mar 26, 2021

Before this change using @tailwind directive in styled-jsx component lead to

styled-jsx-plugin-postcss] postcss failed with TypeError [ERR_INVALID_ARG_TYPE]:
The "url" argument must be of type string. Received type boolean (false)

According to postcss documentation (https://postcss.org/api/#processoptions) the
'from' option should be a string of the filename. We should get this filename as
options.babel.filename (See: https://github.com/vercel/styled-jsx#plugin-options).

Since we don't use the source maps they are now disabled as well.

Before this change using @tailwind directive in styled-jsx component lead to

styled-jsx-plugin-postcss] postcss failed with TypeError [ERR_INVALID_ARG_TYPE]:
The "url" argument must be of type string. Received type boolean (false)

According to postcss documentation (https://postcss.org/api/#processoptions) the
'from' option should be a string of the filename. We should get this filename as
options.babel.filename (See: https://github.com/vercel/styled-jsx#plugin-options).

Since we don't use the source maps they are now disabled as well.
processor.js Outdated

return loaderPromise
.then((plugins) => postcss(plugins).process(src, { from: false }))
.then((plugins) => postcss(plugins).process(src, { from: options.babel.filename, map: false }))
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will break if options.babel is undefined. Can you add a guard?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, I've added a fallback. Just to let you known: I couldn't test it in my setup anymore, since I ended up not relying on this plugin.

johncomposed added a commit to johncomposed/styled-jsx-plugin-postcss that referenced this pull request Dec 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants